Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

different speeds cant be friends. Fix for #585 #599

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

lahrims
Copy link

@lahrims lahrims commented Aug 12, 2024

Fix for Issue #585

  1. Friendships are made with pets of similar speeds - add condition here https://github.com/tonybaloney/vscode-pets/blob/main/src/panel/pets.ts#L108-L110

added a speed threshold and the two pet speeds are compared against the threshould for them to be friends

@lahrims lahrims changed the title diffrent speeds cant be friends diffrent speeds cant be friends. Fix for #585 Aug 12, 2024
@lahrims lahrims changed the title diffrent speeds cant be friends. Fix for #585 different speeds cant be friends. Fix for #585 Aug 19, 2024
src/panel/pets.ts Outdated Show resolved Hide resolved
src/panel/pets.ts Outdated Show resolved Hide resolved
src/panel/pets.ts Outdated Show resolved Hide resolved
Copy link
Owner

@tonybaloney tonybaloney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor changes inline

lahrims and others added 3 commits August 23, 2024 09:14
Co-authored-by: Anthony Shaw <[email protected]>
Co-authored-by: Anthony Shaw <[email protected]>
Co-authored-by: Anthony Shaw <[email protected]>
@lahrims lahrims requested a review from tonybaloney August 23, 2024 14:15
@lahrims
Copy link
Author

lahrims commented Aug 23, 2024

@tonybaloney I see! I should have been more careful about the naming convention

@sharktrexer
Copy link
Contributor

Is there any update on this getting merged? I'm still having this issue that this fixes as of 9/23/2024.

@lahrims
Copy link
Author

lahrims commented Oct 30, 2024

Hi @tonybaloney
Just checking in to see if there is anything more required before this issue is merged. And if there are any other issues, I would love to contribute.

@tonybaloney
Copy link
Owner

Hi @tonybaloney

Just checking in to see if there is anything more required before this issue is merged. And if there are any other issues, I would love to contribute.

Sorry I'm just really busy at the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants